Conversation
gene1wood
commented
Apr 13, 2018
- Take a look at this branch
- Figure out what it does
- Decide if we want to use this
- Merge it or make an issue capturing the idea or close and delete the branch
|
@nnja Would you like to see if there's any unit test content here worth keeping? |
|
@gene1wood Sure. I'll take a look today or tomorrow. |
|
So, I didn't have a very good story for testing agithub. If it were me, I'd rethink this, and restart from scratch. These tests don't mock up the GitHub API (which they probably should). They require connecting to the real API, and thus require entering real credentials for any non-public API usage. |
|
@gene1wood @jpaugh Any chance either of you are at the PyCon 2018 sprints? I was thinking about working on this on Wednesday or for a half day on Thursday. |
|
@nnja I'm not unfortunately. If you want to call out a day and time to meet online I could video chat. |
|
@gene1wood Any chance you're available between 10am and 12pm EST tomorrow? |